Skip to content

Avoid Iceberg diff crash paths in repository tools#95

Merged
Gabriel-Darbord merged 13 commits into
mainfrom
codex/avoid-iceberg-diff-crashes
Jul 1, 2026
Merged

Avoid Iceberg diff crash paths in repository tools#95
Gabriel-Darbord merged 13 commits into
mainfrom
codex/avoid-iceberg-diff-crashes

Conversation

@Gabriel-Darbord

Copy link
Copy Markdown
Member

Summary

  • replace repository export/commit use of workingCopyDiff with direct Tonel export plus Git status/commit
  • write .project metadata during diff-free export so committed checkouts remain attachable
  • replace branch switch/pull/checkout Iceberg diff/adopt paths with Git CLI plus direct package loading/reference updates
  • keep repository operation tests aligned with the new public contract instead of Iceberg internal index state

Verification

  • MCPToolRepositoryOperationTest in Pharo 13: 46 passed, 1 skipped
  • MCPToolRepositoryToolsTest, MCPToolContractsTest, MCPToolStructuredOutputTest: 146 passed
  • MCP-Tests: 640 passed, 1 skipped
  • committed and pushed through the updated repository tool path without transport drop

Addresses #65.

Gabriel-Darbord and others added 2 commits July 1, 2026 13:52
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Gabriel-Darbord and others added 11 commits July 1, 2026 14:23
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
Co-authored-by: Codex <codex@openai.com>
@Gabriel-Darbord Gabriel-Darbord merged commit 10f2550 into main Jul 1, 2026
3 checks passed
@Gabriel-Darbord Gabriel-Darbord deleted the codex/avoid-iceberg-diff-crashes branch July 1, 2026 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant